Skip to content

Fix broken tutorial pages: explorer and zenodo analysis#56

Merged
rdhyee merged 1 commit intoisamplesorg:mainfrom
rdhyee:main
Mar 16, 2026
Merged

Fix broken tutorial pages: explorer and zenodo analysis#56
rdhyee merged 1 commit intoisamplesorg:mainfrom
rdhyee:main

Conversation

@rdhyee
Copy link
Contributor

@rdhyee rdhyee commented Mar 16, 2026

Summary

Fixes two broken tutorial pages on isamples.org:

  • isamples_explorer (PR Fix isamples_explorer: mutable variable for facet summaries #54, already merged): facetSummariesError assigned without mutable keyword in OJS, killing all downstream cells
  • zenodo_isamples_analysis (this PR): SQL queries used old column names (sample_location_latitude, source_collection) but the Jan 2026 wide parquet has different names (latitude, longitude, n). Added column aliases in the view creation.

What works now

  • Dataset overview: 20.7M records, 11.96M with coordinates
  • Source collection table and bar chart
  • Geographic statistics and regional distribution
  • Interactive sampling and world map
  • Viewport map with DuckDB-WASM

Known limitation

  • Material Category Analysis section shows empty — the wide format stores categories as integer arrays (p__has_material_category) not strings. Full fix would require joining back to concept nodes.

Test plan

  • Playwright smoke test: zero Binder Errors (was 57+ before fix)
  • Source collection table renders with correct counts
  • Geographic charts render
  • Manual verification on isamples.org after deploy

🤖 Generated with Claude Code

…rquet

The page queries used old column names (sample_location_latitude,
source_collection) but the Jan 2026 wide parquet has different names
(latitude, longitude, n). Added column aliases in the view creation
so all downstream queries work without rewriting.

Material category analysis section will show empty since the wide
format stores these as integer arrays (p__has_material_category)
rather than string values.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rdhyee rdhyee merged commit 55fabbe into isamplesorg:main Mar 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant